Skip to content

Physiological file download path#1394

Draft
MaximeBICMTL wants to merge 1 commit intoaces:mainfrom
MaximeBICMTL:physio-file-download-path
Draft

Physiological file download path#1394
MaximeBICMTL wants to merge 1 commit intoaces:mainfrom
MaximeBICMTL:physio-file-download-path

Conversation

@MaximeBICMTL
Copy link
Contributor

Sister PR to aces/Loris#10411, add the DbPhysioFile.download_path field and adapt the insertion code to use it.

Also, document the difference between DbPhysioFile.path and DbPhysioFile.download_path in the ORM (I believe we should do that a lot more).

@github-actions github-actions bot added the Language: Python Issue or PR related to the Python codebase label Mar 16, 2026
@MaximeBICMTL MaximeBICMTL changed the title physio file download path Physiological file download path Mar 16, 2026
@MaximeBICMTL MaximeBICMTL added Area: ORM Issue or PR related to the SQLAlchemy integration Complexity: Simple Issue or PR that should be simple to implement, review, or test labels Mar 16, 2026
@MaximeBICMTL MaximeBICMTL force-pushed the physio-file-download-path branch from 2853a41 to 74579d0 Compare March 26, 2026 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: ORM Issue or PR related to the SQLAlchemy integration Complexity: Simple Issue or PR that should be simple to implement, review, or test Language: Python Issue or PR related to the Python codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant